search for: hintonda

...could be enhanced to do that -- though it currently replaces `X->foo() && isa<Y>(X->foo())` with `dyn_cast_or_null<Y>(X->foo())`. Please see: https://reviews.llvm.org/D59802 thanks... don > > ~Craig > > > On Thu, Apr 4, 2019 at 4:37 PM Don Hinton <hintonda at gmail.com> wrote: > >> On Thu, Apr 4, 2019 at 6:29 PM Craig Topper <craig.topper at gmail.com> >> wrote: >> >>> There are a handful of places in LLVM that dosomething like if >>> (dyn_cast_or_null<UndefValue>(P->hasConstantValue())) >&...

Hi Don, On 25 September 2017 at 22:37, Don Hinton <hintonda at gmail.com> wrote: > It'll work in release builds -- just rebuild llvm with LLVM_ENABLE_DUMP > enabled. > That assumes one has control over the LLVM build options. > On Mon, Sep 25, 2017 at 2:35 PM, Dibyendu Majumdar <mobile at majumdar.org.uk> > wrote: >> &gt...

Or, instead of lobbying for llvm to always define dump(), you could lobby for it to enabled by default. On Mon, Sep 25, 2017 at 2:41 PM, Dibyendu Majumdar <mobile at majumdar.org.uk> wrote: > Hi Don, > > On 25 September 2017 at 22:37, Don Hinton <hintonda at gmail.com> wrote: > > It'll work in release builds -- just rebuild llvm with LLVM_ENABLE_DUMP > > enabled. > > > > That assumes one has control over the LLVM build options. > > > > On Mon, Sep 25, 2017 at 2:35 PM, Dibyendu Majumdar < > mobile at m...

On 13 October 2015 at 22:28, don hinton <hintonda at gmail.com> wrote: > However, the build that failed thought it needed to do 106 steps. It hung > on the 83 that didn't exist. Interesting... > Could this be a cmake/ninja issue? It could. Those bots are not cleaning between builds (or it would take hours). If that's what...

...y using the name Chris suggested. Please >> let me know what you think. >> >> https://reviews.llvm.org/D60291 >> >> thanks... >> don >> >> On Thu, Apr 4, 2019 at 2:55 PM David Greene <dag at cray.com> wrote: >> >>> Don Hinton <hintonda at gmail.com> writes: >>> >>> > > if (isa_or_null<T>(var)) { >>> > > ... >>> > > } >>> > > >>> > > at least according to what "isa_or_null" conveys to me. >>> > >>> >...

Hi Don, On 25 September 2017 at 22:29, Don Hinton <hintonda at gmail.com> wrote: > Thanks for reporting this. > > Looks like this one was missed -- the declaration should have been #ifdef'd > away along with the definition. A quick grep indicates there are a number > of them that need to be fixed. > > Here's the original co...

Is there a way to do this with llvm-lit, i.e., use the equivalent of backticks? foo takes a single argument, but doesn't read from stdin. // RUN foo some_arg > %t; FileCheck %s < %t // RUN foo `cat %t` | FileCheck --check-prefix=INVERSE // CHECK: {{^[0-9]+$}} // INVERSE: some_arg thanks... don -------------- next part -------------- An HTML attachment was scrubbed... URL:

It'll work in release builds -- just rebuild llvm with LLVM_ENABLE_DUMP enabled. On Mon, Sep 25, 2017 at 2:35 PM, Dibyendu Majumdar <mobile at majumdar.org.uk> wrote: > Hi Don, > > On 25 September 2017 at 22:29, Don Hinton <hintonda at gmail.com> wrote: > > Thanks for reporting this. > > > > Looks like this one was missed -- the declaration should have been > #ifdef'd > > away along with the definition. A quick grep indicates there are a > number > > of them that need to be fixed. &...

Just posted this fix on ninja's github page, but figured I'd share it with a larger audience. Every time I run cmake && ninja in a new build directory, ninja will rerun cmake because the entry for build.ninja in .ninja_log is older than the timestamp on CMakeCache.txt, even if the timestamps on the actual file isn't older. The following patch fixes the problem, i.e.,

On Sat, Oct 14, 2017 at 4:22 PM, Don Hinton <hintonda at gmail.com> wrote: > Hi Jack: > > The only way I could reproduce the problem you are seeing was to rerun > cmake with -DCMAKE_BUILD_TYPE=Debug in a build directory previously > configured with -DCMAKE_BUILD_TYPE=Release. I can fix that, but not sure > if it's what you a...

The test in question is clang-check-mac-libcxx-fixed-compilation-db.cpp which copies clang-check to a local directory to make sure clang can find libcxx via rpath. However, when built with shared libs, the copy of clang-check can't find any of it's libraries, so I'd like to either turn if off when BUILD_SHARED_LIBS=ON, or find a way to fix it. Thought about trying to use

I've added a patch, temporarily using the name Chris suggested. Please let me know what you think. https://reviews.llvm.org/D60291 thanks... don On Thu, Apr 4, 2019 at 2:55 PM David Greene <dag at cray.com> wrote: > Don Hinton <hintonda at gmail.com> writes: > > > > if (isa_or_null<T>(var)) { > > > ... > > > } > > > > > > at least according to what "isa_or_null" conveys to me. > > > > This is the same convention used by the existing "_or_null...

...__vsnprintf_l hello.obj : error LNK2019: unresolved external symbol ___stdio_common_vfprintf referenced in function __vfprintf_l LINK : error LNK2001: unresolved external symbol _mainCRTStartup hello.exe : fatal error LNK1120: 4 unresolved externals On Sat, Feb 25, 2017 at 11:30 AM, don hinton <hintonda at gmail.com> wrote: > I think you need to tell lli which shared libraries to load, via "-load > option," in order to find missing symbols -- probably libcmt in this case, > since that's what you passed to link in the llc example. > > hth... > Don > > On Fr...

On 11 June 2017 at 14:04, don hinton <hintonda at gmail.com> wrote: > Assuming you know it's a 64 bit value, and the function you are calling > takes a uint64_t, try this: > The values from the test program are of type: i32/i32*/i32**. Can't I interpret these as uint64_t some way? > > Value* args[] = {store_inst-&g...

On Sat, Oct 14, 2017 at 11:38 AM, Don Hinton <hintonda at gmail.com> wrote: > Sorry, LLVM_FORCE_ENABLE_DUMP is the variable, which is used along with > LLVM_ENABLE_ASSERTIONS to set LLVM_ENABLE_DUMP. > > Again, I'm working on a fix, but could you also provide your cmake > command? Looks like we aren't handling your use case...

On Thu, Apr 4, 2019 at 10:30 AM David Greene <dag at cray.com> wrote: > I don't think that's a correct replacement. > > if (var && isa<T>(var)) { > ... > } > > is not the same as: > > if (isa_or_null<T>(var)) { > ... > } > > at least according to what "isa_or_null" conveys to me. > This is the same

On Sat, Oct 14, 2017 at 11:25 AM, Don Hinton <hintonda at gmail.com> wrote: > Hi Jack: > > Yes, I was just looking at that. Seems like TableGen wasn't done along > with the rest of llvm. I'll work up a complete patch shortly. > > Btw, I'm curious how this happened. Do you have a stale CMakeCache.txt by > any chan...

On Sun, Oct 15, 2017 at 12:40 PM, Don Hinton <hintonda at gmail.com> wrote: > On Sun, Oct 15, 2017 at 8:34 AM, Aaron Ballman <aaron at aaronballman.com> > wrote: >> >> FWIW, most of the ones I was fixing up were guarded by NDEBUG instead >> of LLVM_ENABLE_DUMP. Switching to LLVM_ENABLE_DUMP fixed the link >> error...

If your cut-n-paste is accurate, you seem to be missing a closing quote for CMAKE_CXX_LINK_FLAGS. hth... don On Jul 8, 2015 2:22 AM, "David A. Greene" <greened at obbligato.org> wrote: > greened at obbligato.org (David A. Greene) writes: > > > I looked at the instructions on the libc++ page and for Linux it > > recommends building with -stdlib=libc++ -lc++abi.

On Sun, Oct 15, 2017 at 10:58 AM, Don Hinton <hintonda at gmail.com> wrote: > Thanks Aaron. > > I don't have a Windows system, and haven't seen any buildbot failures, so I > it's difficult to come up with a patch for something I can't reproduce > locally or see any actual failures. Could you send me the commands you u...