search for: reinterpretcast

On Wed, Jan 22, 2020 at 4:40 PM David Greene <greened at obbligato.org> wrote: > Hubert Tong <hubert.reinterpretcast at gmail.com> writes: > > > The update process in Phab in rather manual and it does lead to more > noise. > > The GitHub force push risks loss of context for earlier comments (not > just > > in terms of display, like viewing older comments on a later diff in > Phab)...

Hubert Tong <hubert.reinterpretcast at gmail.com> wrote on 10.06.2020 23:51:54: > From: Hubert Tong <hubert.reinterpretcast at gmail.com> > To: Kai Peter Nacke <kai.nacke at de.ibm.com> > Cc: llvm-dev <llvm-dev at lists.llvm.org> > Date: 10.06.2020 23:52 > Subject: [EXTERNAL] Re: [llvm-dev] RFC:...

...ing is changed in the correct way. One function related >> to >> this has already been mentioned: getMostGenericTBAA. >> >> Later, >> Steven Perron >> >> >> >> From: Hubert Tong <hubert.reinterpretcast at gmail.com >> <mailto:hubert.reinterpretcast at gmail.com> >> <mailto:hubert.reinterpretcast at gmail.com >> <mailto:hubert.reinterpretcast at gmail.com>>> >> To: Steven Perron/Toronto/IBM at IBMCA >&gt...

...ility to make sure the >> aliasing is changed in the correct way. One function related to >> this has already been mentioned: getMostGenericTBAA. >> >> Later, >> Steven Perron >> >> >> >> From: Hubert Tong <hubert.reinterpretcast at gmail.com >> <mailto:hubert.reinterpretcast at gmail.com>> >> To: Steven Perron/Toronto/IBM at IBMCA >> Cc: Daniel Berlin <dberlin at dberlin.org >> <mailto:dberlin at dberlin.org>>, llvm-dev <llvm-dev at lists.llvm.or...

...en-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#1787), this isn't valid because unspecified value is a 'stable' value. In our suggestion, well-defined bits are stored into padding at object creation, so it becomes okay. Juneyoung On Fri, Oct 9, 2020 at 11:58 PM Hubert Tong < hubert.reinterpretcast at gmail.com> wrote: > On Thu, Oct 8, 2020 at 11:54 PM Juneyoung Lee <juneyoung.lee at sf.snu.ac.kr> > wrote: > >> // Members are initialized to poison at object creation. >>>> p = alloca {i8, i32} // p[0], p[4~7] are poison >>>> p[0] is an i8, so it...

On Sun, 13 Sep 2020 at 15:51, Hubert Tong <hubert.reinterpretcast at gmail.com> wrote: > If you mean "amend" the message in the Web UI before merging, then yes, >> they let you change the message, but it's very easy to forget to do >> that. > > That's what I meant, yes. "Easy to forget" generally goes away when...

On Thu, Jan 23, 2020 at 11:37 AM David Greene <greened at obbligato.org> wrote: > Hubert Tong <hubert.reinterpretcast at gmail.com> writes: > > >> I read this as the refresh being an entirely new GitHub PR. Is that > >> right? Normally I would expect the same PR to be used but the rebase > >> would cause a force-push of the branch which would update the PR with > >> the...

...lue in the C++14 draft in the past, but it is removed: http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#1787 The removed phrase did not exist in C++11, so I believe it is fine to use poison for uninitialized char types. Juneyoung On Fri, Oct 9, 2020 at 12:19 PM Hubert Tong < hubert.reinterpretcast at gmail.com> wrote: > On Thu, Oct 8, 2020 at 11:09 PM Juneyoung Lee <juneyoung.lee at sf.snu.ac.kr> > wrote: > >> It is UB when a poison is passed to certain operations that raise UB on >> poison, such as division by poison/dereferencing poison pointer/branching >...

On Thu, Jan 16, 2020 at 1:17 PM David Greene <greened at obbligato.org> wrote: > Hubert Tong <hubert.reinterpretcast at gmail.com> writes: > > >> I can see how having multiple patches up at once for review might speed > >> things up. But what if a very first patch in a series needs major > >> changes and that affects every single following patch? The series has > >> to...

On Fri, Nov 25, 2016 at 1:42 AM, Daniel Berlin <dberlin at dberlin.org> wrote: > What is the purpose of the union there? > The purpose of the union is to increase portability by ensuring that the placement new is not being performed on insufficiently sized or aligned memory. > I ask because pretty much no compiler will respecting the unioning without > visible accesses in all

On Thu, Jan 17, 2019 at 6:51 PM Hubert Tong < hubert.reinterpretcast at gmail.com> wrote: > I'm wondering if this would wreak havoc upon semi-case-sensitive systems > like Windows. I'd like to hear from people who use Git and SVN on Windows. > FWIW, I will of course not do this in any case where there might be a collision due to such file syste...

...d job. > > best regards, > Fedor. > > On 1/23/20 3:02 AM, Hubert Tong via llvm-dev wrote: > > On Wed, Jan 22, 2020 at 5:19 PM David Greene <greened at obbligato.org > > <mailto:greened at obbligato.org>> wrote: > > > > Hubert Tong <hubert.reinterpretcast at gmail.com > > <mailto:hubert.reinterpretcast at gmail.com>> writes: > > > > > It was already mentioned earlier in the thread that: > > > - The individual commits cannot be approved in the sense of > > approving a PR. > > > &gt...

...32} // p[0], p[4~7] are poison q = alloca {i8, i32} // we want to copy p to q v = load i8* p[0] // v is poison store i8 v, i8* q[0] // poison is simply copied; no UB happened Similarly, passing/returning poison is allowed as well. Juneyoung On Fri, Oct 9, 2020 at 10:45 AM Hubert Tong < hubert.reinterpretcast at gmail.com> wrote: > On Thu, Oct 8, 2020 at 7:13 PM Juneyoung Lee <juneyoung.lee at sf.snu.ac.kr> > wrote: > >> > It is important to note that this applies to trap representations and >> not to unspecified values. A structure or union never has a trap >> re...

...we can't prove > the call behaves the same with errno, then give up. > I'm not sure where the select / branching happens, but I don't see that > happening in the initial transform (called from -instcombine) > > On Mon, Sep 14, 2020 at 12:58 PM Hubert Tong < > hubert.reinterpretcast at gmail.com> wrote: > >> On Mon, Sep 14, 2020 at 12:45 PM Sanjay Patel <spatel at rotateright.com> >> wrote: >> >>> Yes, that looks like a bug. The transform is ok in general for negative >>> numbers, but -Inf is a special-case for pow(), right? >...

On Wed, Jan 22, 2020 at 5:19 PM David Greene <greened at obbligato.org> wrote: > Hubert Tong <hubert.reinterpretcast at gmail.com> writes: > > > It was already mentioned earlier in the thread that: > > - The individual commits cannot be approved in the sense of approving a > PR. > > Later conversation makes me wonder how often that really happens though. > In a true patch series I wo...

> On Jan 23, 2020, at 08:37, David Greene via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > Hubert Tong <hubert.reinterpretcast at gmail.com> writes: > >>> I read this as the refresh being an entirely new GitHub PR. Is that >>> right? Normally I would expect the same PR to be used but the rebase >>> would cause a force-push of the branch which would update the PR with >>> the new...

On Mon, Feb 13, 2017 at 10:07 AM, Hubert Tong < hubert.reinterpretcast at gmail.com> wrote: > On Mon, Feb 13, 2017 at 2:23 AM, Daniel Berlin via llvm-dev < > llvm-dev at lists.llvm.org> wrote: > >> >>> I don't think this fully solves the problem -- you'll also need to fix >>> getMostGenericTBAA. That is, even if you...

On Tue, Feb 14, 2017 at 5:51 AM, Hubert Tong < hubert.reinterpretcast at gmail.com> wrote: > On Mon, Feb 13, 2017 at 10:39 PM, Daniel Berlin <dberlin at dberlin.org> > wrote: > >> >> >> On Mon, Feb 13, 2017 at 10:07 AM, Hubert Tong < >> hubert.reinterpretcast at gmail.com> wrote: >> >>> On Mon, Feb 13, 2...

On Tue, Oct 31, 2017 at 6:26 PM, Zachary Turner via llvm-dev < llvm-dev at lists.llvm.org> wrote: > > On Tue, Oct 31, 2017 at 3:19 PM Justin Bogner <mail at justinbogner.com> > wrote: > >> Zachary Turner <zturner at google.com> writes: >> >> > If 3 months later we started requiring C++17, someone could build clang >> 6 >> > with

On Fri, 19 Jun 2020, 18:55 Hubert Tong, <hubert.reinterpretcast at gmail.com> wrote: > On Fri, Jun 19, 2020 at 12:32 PM Anton Korobeynikov via llvm-dev < > llvm-dev at lists.llvm.org> wrote: > >> Just my 2 cents here: we are working on enabling this as a part of >> bugzilla migration as PRs and issues are very tied inside GitHub....