[PATCH] [lld] findAbsoluteAtom asserts the atom is really existing

Rui Ueyama ruiu at google.com
Wed Feb 11 16:46:58 PST 2015


I agree with Simon. Adding a boolean flag to the existing function does not sounds like a good idea, just because findAbsoluteAtom(s, true /* dieNotFound */) doesn't look better than getAbsoluteAtom(s).


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7549

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list