[libcxx-commits] [PATCH] D153968: [libc++] Stop using __builtin_assume in _LIBCPP_ASSERT

[Konstantin Varlamov via Phabricator via libcxx-commits]

var-const accepted this revision.
var-const added a comment.

FWIW, I also found the information in the thread completely counterintuitive. I guess I'll be thinking of the "assume" attribute kinda like "inline", only beneficial in certain circumstances (at least for now). But given the thread, this patch looks like the right thing to do, so LGTM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D153968/new/

https://reviews.llvm.org/D153968